Skip to content

New package: radicle-1.9.1#60993

Open
gnist-dev wants to merge 1 commit into
void-linux:masterfrom
gnist-dev:radicle
Open

New package: radicle-1.9.1#60993
gnist-dev wants to merge 1 commit into
void-linux:masterfrom
gnist-dev:radicle

Conversation

@gnist-dev

Copy link
Copy Markdown
Contributor

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64-musl)
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • armv6l-musl

Comment thread srcpkgs/radicle/template Outdated
mkdir -p ${XBPS_SRCDISTDIR}/radicle-${version}
cd ${XBPS_SRCDISTDIR}/radicle-${version}
curl --output radicle-${version}.tar.gz \
https://seed.radicle.dev/raw/rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5/archive/releases/${version}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a distfile that is checksum verified.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've changed to use the distfiles (.tar.gz) from upstream and fixed my previous issue that made me do the 'curl' stuff by putting in a post_fetch instead.

@jonpikum

Copy link
Copy Markdown

Last attempt (#52527) also packaged radicle-httpd.
(I can't remember if it's required or not).
I assume the patch isn't needed anymore.

@tranzystorekk tranzystorekk added the new-package This PR adds a new package label Jun 11, 2026
@gnist-dev

Copy link
Copy Markdown
Contributor Author

Last attempt (#52527) also packaged radicle-httpd. (I can't remember if it's required or not). I assume the patch isn't needed anymore.

I just looked through the crates of the repo, and it seems upstream has split the http-UI out into it's own separate repo named radicle-explorer instead.

Comment thread srcpkgs/radicle/template Outdated
Comment on lines +18 to +26
post_fetch() {
cd ${XBPS_SRCDISTDIR}/radicle-${version}
mv ${version} radicle-${version}.tar.gz
}

do_extract() {
mkdir -p ${XBPS_BUILDDIR}/radicle-${version}
bsdtar xf ${XBPS_SRCDISTDIR}/radicle-${version}/radicle-${version}.tar.gz -C ${XBPS_BUILDDIR}/radicle-${version}
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

None of this should be required (post_fetch, do_extract).

Comment thread srcpkgs/radicle/template Outdated
license="MIT AND Apache-2.0"
homepage="https://radicle.dev"
changelog="https://radicle.network/nodes/seed.radicle.dev/rad%3Az3gqcJUoA1n9HaHKufZs5FCSGazv5/tree/CHANGELOG.md"
distfiles="https://seed.radicle.dev/raw/rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5/archive/releases/${version}"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
distfiles="https://seed.radicle.dev/raw/rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5/archive/releases/${version}"
distfiles="https://seed.radicle.dev/raw/rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5/archive/releases/${version}>${version}.tar.gz"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's nifty that one can do output redirection in the distfiles directly, I'll keep that in mind for the future. I've removed the unnecessary post_fetch and do_extract and added the redirection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-package This PR adds a new package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants